-
Notifications
You must be signed in to change notification settings - Fork 842
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding more Add data logos to Icons #341
Conversation
Adding Docker, Kubernetes and Redis logos
@@ -112,7 +112,6 @@ exports[`EuiCodeEditor is rendered 1`] = ` | |||
wrap="off" | |||
/> | |||
<div | |||
aria-hidden="true" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@snide Not sure why these are removed, it happens after I run tests.
@@ -17,6 +17,7 @@ exports[`EuiFormRow behavior onBlur is called in child 1`] = ` | |||
<EuiFormLabel | |||
htmlFor="generated-id" | |||
isFocused={false} | |||
isInvalid={undefined} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@snide Same goes for these, added after tests.
You're going to want to clear your cache.
Then rerun the tests. |
@snide Right, will do |
@snide Odd. The tests pass, but it seems it failed anyways 😟 |
Merging. Travis acting up again. |
Closes elastic/kibana#16279
Adding Docker, Kubernetes and Redis logos to the Icons/Logos list for the Add data UI in Kibana.